Skip to content

feat(test-suite): add per-service override support to fhevm-cli#2075

Merged
Eikix merged 2 commits intocodex/bun-fhevm-clifrom
feat/test-suite-per-service-override
Mar 10, 2026
Merged

feat(test-suite): add per-service override support to fhevm-cli#2075
Eikix merged 2 commits intocodex/bun-fhevm-clifrom
feat/test-suite-per-service-override

Conversation

@PanGan21
Copy link
Copy Markdown
Contributor

Summary

  • Extend --override to accept per-service filtering (--override coprocessor:host-listener,tfhe-worker), building only specified services locally while pulling the rest from the registry at their resolved tags
  • Services sharing a Docker image are automatically co-selected (e.g., host-listener includes host-listener-poller)
  • Invalid service names in overrides produce errors with the list of valid suffixes
  • Non-overridden services in the same group now get compatibility args applied individually instead of all-or-nothing
  • Warns user when trying to override services that share the same database

Related: https://github.com/zama-ai/fhevm-internal/issues/1042

@PanGan21 PanGan21 requested a review from a team as a code owner March 10, 2026 10:49
@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Mar 10, 2026

Thank you for your pull request. We require contributors to sign our Contributor License Agreement / Terms and Conditions, and we don't seem to have the users @PanGan21 on file. In order for us to review and merge your code, please sign:

  • For individual contribution: our CLA
  • for Bounty submission, if you are an individual: our T&C
  • for Bounty submission, if you are a company: our T&C
    to get yourself added.

If you already signed one of this document, just wait to be added to the bot config.

@mergify
Copy link
Copy Markdown

mergify bot commented Mar 10, 2026

🧪 CI Insights

Here's what we observed from your CI run for a726e58.

🟢 All jobs passed!

But CI Insights is watching 👀

@PanGan21 PanGan21 force-pushed the feat/test-suite-per-service-override branch from b9867ca to a726e58 Compare March 10, 2026 10:55
@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Mar 10, 2026

Thank you for your pull request. We require contributors to sign our Contributor License Agreement / Terms and Conditions, and we don't seem to have the users @PanGan21 on file. In order for us to review and merge your code, please sign:

  • For individual contribution: our CLA
  • for Bounty submission, if you are an individual: our T&C
  • for Bounty submission, if you are a company: our T&C
    to get yourself added.

If you already signed one of this document, just wait to be added to the bot config.

@Eikix Eikix merged commit 2c41110 into codex/bun-fhevm-cli Mar 10, 2026
53 of 54 checks passed
@Eikix Eikix deleted the feat/test-suite-per-service-override branch March 10, 2026 11:12
Eikix pushed a commit that referenced this pull request Mar 10, 2026
* feat(test-suite): add per-service override support to fhevm-cli

* feat(test-suite): warn user when selecting services that share a database
Eikix pushed a commit that referenced this pull request Mar 11, 2026
* feat(test-suite): add per-service override support to fhevm-cli

* feat(test-suite): warn user when selecting services that share a database
Eikix pushed a commit that referenced this pull request Mar 17, 2026
* feat(test-suite): add per-service override support to fhevm-cli

* feat(test-suite): warn user when selecting services that share a database
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants